-
Notifications
You must be signed in to change notification settings - Fork 32
⬆️Upgrade to UV 0.7, pre-commit tools and dump installation of pip/setuptools/wheels #8000
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR upgrades the UV dependency from version 0.6 to 0.7 across many services, updates pre-commit tool revisions, and removes the pip upgrade commands from the Makefile.
- Update UV_VERSION to "0.7" in multiple Dockerfiles
- Bump pre-commit hook revisions for pyupgrade and isort
- Remove the inline pip upgrade commands in the Makefile
Reviewed Changes
Copilot reviewed 29 out of 29 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| services/web/Dockerfile | Updated UV_VERSION to "0.7" |
| services/storage/Dockerfile | Updated UV_VERSION to "0.7" |
| services/resource-usage-tracker/Dockerfile | Updated UV_VERSION to "0.7" |
| services/payments/Dockerfile | Updated UV_VERSION to "0.7" |
| services/notifications/Dockerfile | Updated UV_VERSION to "0.7" |
| services/migration/Dockerfile | Updated UV_VERSION to "0.7" |
| services/invitations/Dockerfile | Updated UV_VERSION to "0.7" |
| services/efs-guardian/Dockerfile | Updated UV_VERSION to "0.7" |
| services/dynamic-sidecar/Dockerfile | Updated UV_VERSION to "0.7" |
| services/dynamic-scheduler/Dockerfile | Updated UV_VERSION to "0.7" |
| services/director/Dockerfile | Updated UV_VERSION to "0.7" |
| services/director-v2/Dockerfile | Updated UV_VERSION to "0.7" |
| services/datcore-adapter/Dockerfile | Updated UV_VERSION to "0.7" |
| services/dask-sidecar/Dockerfile | Updated UV_VERSION to "0.7" |
| services/clusters-keeper/Dockerfile | Updated UV_VERSION to "0.7" |
| services/catalog/Dockerfile | Updated UV_VERSION to "0.7" |
| services/autoscaling/Dockerfile | Updated UV_VERSION to "0.7" |
| services/api-server/Dockerfile | Updated UV_VERSION to "0.7" |
| services/agent/Dockerfile | Updated UV_VERSION to "0.7" |
| scripts/pydeps-docker/Dockerfile | Updated UV_VERSION to "0.7" |
| scripts/openapi/oas_resolver/Dockerfile | Updated UV_VERSION to "0.7" |
| scripts/maintenance/migrate_project/Dockerfile | Updated UV_VERSION to "0.7" |
| scripts/erd/Dockerfile | Updated UV_VERSION to "0.7" |
| requirements/tools/Dockerfile | Updated UV_VERSION to "0.7" |
| packages/service-integration/Dockerfile | Updated UV_VERSION to "0.7" |
| packages/postgres-database/scripts/erd/Dockerfile | Updated UV_VERSION to "0.7" |
| packages/postgres-database/docker/Dockerfile | Updated UV_VERSION to "0.7" |
| Makefile | Removed pip upgrade commands for uv-related tools |
| .pre-commit-config.yaml | Upgraded pyupgrade and isort hook revisions |
matusdrobuliak66
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 👍
|
@mergify queue |
🟠 Waiting for conditions to match
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #8000 +/- ##
===========================================
+ Coverage 77.45% 88.08% +10.63%
===========================================
Files 1540 1565 +25
Lines 60706 62128 +1422
Branches 1103 623 -480
===========================================
+ Hits 47017 54727 +7710
+ Misses 13366 7236 -6130
+ Partials 323 165 -158
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
odeimaiz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unacceptable
come on, it happens to the best of us |
|
@mergify queue |
🟠 Waiting for conditions to match
|
pcrespov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thx!
|
@mergify queue |
🟠 Waiting for conditions to match
|
|
@mergify requeue |
☑️ This pull request is already queued |
|



What do these changes do?
This pull request includes updates to dependency versions and Dockerfile configurations across multiple services and tools. The most important changes involve upgrading the
UV_VERSIONargument in Dockerfiles and updating pre-commit hooks for code formatting and sorting.Dependency Version Updates:
.pre-commit-config.yaml: Updatedpyupgradeto versionv3.20.0andisortto version6.0.1. These are used for code upgrades and import sorting, respectively. [1] [2]Dockerfile Configuration Updates:
UV_VERSIONfrom0.6to0.7across multiple Dockerfiles, includingpackages/postgres-database/docker/Dockerfile,services/api-server/Dockerfile, and others. This ensures all services and tools use the latest version of the UV base image. [1] [2] [3] and others)Makefileunder the.venvtarget, simplifying the virtual environment creation process.Related issue/s
How to test
Dev-ops